-
Notifications
You must be signed in to change notification settings - Fork 706
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Mothur 1.48.2 for foss-2023b #21857
base: develop
Are you sure you want to change the base?
Add Mothur 1.48.2 for foss-2023b #21857
Conversation
Updated software
|
@@ -0,0 +1,118 @@ | |||
diff -ruN mothur-1.48.2.orig/Makefile /tmp/eb/Mothur/1.48.2/foss-2023b/mothur-1.48.2/Makefile |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We ask that patches include a short description what it solves and their author for reference.
Once you include this, please also include the checksum for the patch in the easyconfig.
You can use EB to inject the checksums automatically using
eb Mothur-1.48.2-foss-2023b.eb --inject-checksums
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
# MOTHUR_FILES - The MOTHUR_FILES parameter is optional, but allows you to set a default location for mothur to look for input files it can't find. This is often used for reference files you want to store in one location separate from your data. | ||
|
||
-INSTALL_DIR ?= "\"Enter_your_mothur_install_path_here\"" | ||
+INSTALL_DIR ?= "%(installdir)s" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm, is this tested to work? I don't see how it could, the %(installdir)s templates are not applied to patches.
Looking at the custom easyblock that this thing unfortunately needs to use, it's just manually making the installation via copying files, so I this probably doesn't even have any impact.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was having problems getting the install to work without this change. Before changing this I was getting "undefined reference to main" and others as errors.
So if I understand what you are saying it would mean that I am just setting this to empty and thus could try like in the "MOTHUR_FILES" variable?
OBJECTS+=$(patsubst %.cpp,%.o,$(wildcard *.cpp)) | ||
OBJECTS+=$(patsubst %.c,%.o,$(wildcard *.c)) | ||
|
||
Binary files mothur-1.48.2.orig/source/calculators/prng.o and /tmp/eb/Mothur/1.48.2/foss-2023b/mothur-1.48.2/source/calculators/prng.o differ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this looks like something that shouldn't be included in the patch
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok thanks, updated here: 6104290
+++ /tmp/eb/Mothur/1.48.2/foss-2023b/mothur-1.48.2/source/uchime_src/makefile 2024-11-15 10:29:38.800547895 +0100 | ||
@@ -1,4 +1,4 @@ | ||
-CXXFLAGS = -O3 -D_FILE_OFFSET_BITS=64 -DNDEBUG=1 -DUCHIMES=1 | ||
+CXXFLAGS = -O3 -D_FILE_OFFSET_BITS=64 -DNDEBUG=1 -DUCHIMES=1 -std=c++11 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm, i wonder if we can just make this add the -D flags instead of overwriting CXXFLAGS completely, so that it catches the EB environment variables which set compiler flags (though I'm not going to insist of it in any way gets tricky).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very sorry, but I'm not sure how to do this. This would be in the .eb file? thanks
Hello,
Thanks to the help of @Micket, this is an attempt to resolve #21849.
This is my first PR for Easybuild and happy to adapt as much as needed if this is a helpful addition.
Thanks, Julia